Skip to content

fix(ci): classify *.md-only diffs as docs-only + skill guidance for ci:skip labels (#611)#613

Merged
rickylabs merged 5 commits into
mainfrom
fix/611-ci-docs-only-md
Jul 11, 2026
Merged

fix(ci): classify *.md-only diffs as docs-only + skill guidance for ci:skip labels (#611)#613
rickylabs merged 5 commits into
mainfrom
fix/611-ci-docs-only-md

Conversation

@rickylabs

@rickylabs rickylabs commented Jul 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

Classifies repository-wide Markdown/MDX-only diffs as docs-only while preserving rename-aware conservative handling and explicit full-CI overrides. Adds proactive CI skip-label guidance to the PR/harness skills and regenerates Claude mirrors.

Scope

Slices

  • S1 Harness research, locked plan, and Design checkpoint — 5486637b
  • S2 Classifier contract, implementation, and regressions — 34b33781
  • S3 PR/harness skill guidance and generated Claude mirrors — 9279e40c
  • Separate-session IMPL-EVAL trail — 5b44d7b1

Definition of Done

  • Markdown/MDX-only diffs under any directory classify docs-only.
  • Mixed source and critical Deno/workflow paths force full classification.
  • Rename-aware regression coverage passes.
  • Source skill guidance proactively applies CI skip labels with ci:full as escape hatch.
  • Required tests, scoped check, mirror check, and lock-hygiene check pass.
  • Separate-session IMPL-EVAL passes.

Validation

  • deno test --no-lock -A .github/scripts/ — PASS, 30 passed / 0 failed
  • scoped run-deno-check.ts --root .github/scripts --ext ts,tsx — PASS, 2 files / 0 findings
  • scoped format and evaluator lint — PASS, 0 findings
  • deno task agentic:sync-claude:check — PASS, 17 skills / 21 mirrored files
  • git diff -- deno.lock — empty
  • separate Claude Opus IMPL-EVAL — PASS

Harness

  • Run dir: .llm/runs/fix-611-ci-docs-only-md--ci-classifier/
  • Phase: close / ready-merge
  • PLAN-EVAL and IMPL-EVAL both passed in separate opposite-family sessions.

Drift / Debt

  • The requested status:in-progress label does not exist in the repository taxonomy. The PR followed the canonical single-status lifecycle and moves to status:ready-merge after IMPL-EVAL PASS.
  • The PR skill's old 0.0.1-beta.1 milestone target is absent from the live repository. This beta.6 follow-up is assigned to the next open beta milestone, 0.0.1-beta.7.
  • No architecture debt created.

@rickylabs rickylabs added this to the 0.0.1-beta.7 milestone Jul 11, 2026 — with ChatGPT Codex Connector
@rickylabs ChatGPT Codex Connector

rickylabs commented Jul 11, 2026

Copy link
Copy Markdown
Owner Author

[PHASE: PLAN-EVAL] [VERDICT: APPROVED]

Separate local Claude Opus PLAN-EVAL passed every Plan-Gate item. It spot-checked the current package-Markdown denylist behavior and rename-aware dual-path parsing, and found no open decision that would force rework.

Evidence

  • Verdict artifact: .llm/runs/fix-611-ci-docs-only-md--ci-classifier/plan-eval.md
  • Commit: dc1dd3b5
  • Implementation remained untouched until this PASS.

Next

  • Land classifier regressions/logic, then skill guidance and generated mirrors.

@rickylabs ChatGPT Codex Connector

rickylabs commented Jul 11, 2026

Copy link
Copy Markdown
Owner Author

[PHASE: IMPL]

Classifier slice landed and was substantively reviewed against the approved precedence/rename contract.

Slice 2

  • Commit: 34b33781
  • Markdown/MDX now wins across ordinary package/plugin/app prefixes only after explicit workflow/Deno critical paths are rejected.
  • Rename/copy parsing still classifies both old and new paths.
  • Requested regressions added: package Markdown-only, Markdown + TS, Markdown + deno.lock, and package Markdown rename.

Gates

  • deno test --no-lock -A .github/scripts/ — PASS, 30/30
  • scoped check — PASS, 2 files / 0 findings
  • scoped format check — PASS, 2 files / 0 findings
  • deno.lock — unchanged

@rickylabs rickylabs added status:impl and removed status:plan-eval Awaiting plan evaluation labels Jul 11, 2026 — with ChatGPT Codex Connector
@rickylabs ChatGPT Codex Connector

rickylabs commented Jul 11, 2026

Copy link
Copy Markdown
Owner Author

[PHASE: IMPL]

Skill-guidance slice landed and generated mirrors are in sync.

Slice 3

  • Commit: 9279e40c
  • netscript-pr: docs-only PRs proactively get ci:skip-e2e and, when applicable, ci:skip-scaffold; ci:full wins as escape hatch.
  • netscript-harness: opening-session guidance records the intentional cheap lane.
  • .claude/skills regenerated only with deno task agentic:sync-claude.

Gates

  • deno task agentic:sync-claude:check — PASS, 17 skills / 21 mirrored files
  • final classifier tests — PASS, 30/30
  • final scoped check — PASS
  • docs-overlay source/link/terminology review — PASS
  • git diff --check and lock hygiene — PASS

This PR itself changes TypeScript, so docs-only skip labels are intentionally not applied.

@rickylabs rickylabs mentioned this pull request Jul 11, 2026
Closed
2 tasks
@rickylabs ChatGPT Codex Connector

rickylabs commented Jul 11, 2026

Copy link
Copy Markdown
Owner Author

[PHASE: IMPL-EVAL] [VERDICT: PASS]

Separate local Claude Opus evaluation independently verified the approved scope and close-gate.

Evidence

Findings

  • No blocking findings.
  • Optional future cleanup: deduplicate critical basenames between classifier helpers.

Next

  • PR body/DoD refreshed; transition to status:ready-merge and ready-for-review.

@rickylabs
rickylabs marked this pull request as ready for review July 11, 2026 07:56
@rickylabs
rickylabs merged commit 65ccd2a into main Jul 11, 2026
10 of 12 checks passed
@rickylabs rickylabs mentioned this pull request Jul 11, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area:cli priority:p2 Medium status:ready-merge type:fix

Projects

None yet

Milestone

0.0.1-beta.7

Development

Successfully merging this pull request may close these issues.

ci: classify packages/**/*.md-only diffs as docs-only + harness guidance to apply ci:skip labels on doc PRs

1 participant

@rickylabs